Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport parts of EventPipe PR 43711 not currently in C library. #46214

Conversation

lateralusX
Copy link
Member

Backport of missing parts from #43711 , parts of this was already fixed in EventPipe C library when initially detected and reported as and issue in C++ library.

@lateralusX
Copy link
Member Author

/CC @josalem

@danmoseley
Copy link
Member

Our systems assume issues and PR's have exactly one area-XX label. If this applies to both, does area-System.Diagnostics.Tracing work well enough?

@josalem
Copy link
Contributor

josalem commented Dec 20, 2020

I don't think area-System.Diagnostics.Tracing matches for this. Now that we're combining the EventPipe implementation for mono and coreclr maybe we should add an area-tracing that isn't specific to mono or coreclr?

@danmoseley
Copy link
Member

I'm fine with that, your call. If you do make one you'll need to go back and relabel older issues as appropriate so that we can retrain the bot and it can apply it.

@josalem
Copy link
Contributor

josalem commented Jan 6, 2021

For now, I'm going to consolidate these changes under area-tracing-coreclr and we can add a new label later.

@lateralusX lateralusX merged commit 6b606df into dotnet:master Jan 7, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Feb 6, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants